Skip to content

Conversation

@cameronangliss
Copy link
Contributor

@cameronangliss cameronangliss commented Jan 2, 2026

Improved move tracking guided by strict integration tests, which now also validate the tracked battle state against the "active" section of the request message we receive. I also remove the dead code of EmptyMove, which is completely unused throughout poke-env.

@cameronangliss cameronangliss changed the title Track moves Fix move tracking Jan 2, 2026
@codecov
Copy link

codecov bot commented Jan 2, 2026

Codecov Report

❌ Patch coverage is 75.50336% with 73 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.55%. Comparing base (f458350) to head (5d9b556).
⚠️ Report is 246 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #816      +/-   ##
==========================================
+ Coverage   83.38%   83.55%   +0.17%     
==========================================
  Files          39       47       +8     
  Lines        3918     5754    +1836     
==========================================
+ Hits         3267     4808    +1541     
- Misses        651      946     +295     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cameronangliss cameronangliss marked this pull request as ready for review January 5, 2026 21:31
@cameronangliss cameronangliss force-pushed the track-moves branch 3 times, most recently from cadfaae to 75a1287 Compare February 8, 2026 03:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants